Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs(discover): add phalcode url to discover page #2878

Merged
merged 2 commits into from
Oct 23, 2023
Merged

docs(discover): add phalcode url to discover page #2878

merged 2 commits into from
Oct 23, 2023

Conversation

Alfagun74
Copy link
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Docs
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Shows Phalcode Logo on Discover Page

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

Gamevault and LetsWatchThis are both backed by your awesome framework. Thank you! 💖

@jmcdo29
Copy link
Member

jmcdo29 commented Oct 11, 2023

As there's already many logos on the page, for the time being we've decided to only add links to companies using Nest, to keep the page loading from slowing down the experience for the user. If you'd like to remove the logo, and just keep the link added to the list, that would be great

@Alfagun74
Copy link
Contributor Author

@jmcdo29 sure, i removed the image

@kamilmysliwiec kamilmysliwiec changed the title docs(discover): add phalcode logo to discover page docs(discover): add phalcode url to discover page Oct 23, 2023
@kamilmysliwiec kamilmysliwiec merged commit 7f393a9 into nestjs:master Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants